Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Anisotropic Sampling option to ZZZ GSP #703

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Cryotechnic
Copy link
Collaborator

AKA Anisotropic Filtering, but kept the in-game name for clarity. All presets seem to not change the value, keeping it at x8.

Main Goal

Update ZZZ GSP so that it is in parity with the in-game options.

PR Status :

  • Overall Status : Done ✅
  • Commits : Done ✅
  • Synced to base (Collapse:main) : Yes ✅
  • Build status : OK ✅
  • Crashing : No

Changes:

  • [New] Added new dropdown menu for Anisotropic Sampling option (x1, x2, x4, x8, x16), with x8 as default
    • [Loc] Added new i18n string for option

AKA Anisotropic Filtering, but kept the in-game name for clarity. All presets seem to not change the value, keeping it at x8.
@Cryotechnic Cryotechnic requested a review from bagusnl March 16, 2025 20:03
@Cryotechnic Cryotechnic self-assigned this Mar 16, 2025
Copy link

Copy link
Member

@bagusnl bagusnl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR ack'd

These are just some regular comments, but I think for the localization one you can reuse instead of remaking the entry

/// Sets the in-game quality settings for Anisotropic Sampling
/// </summary>
/// <see cref="AnisotropicSamplingOption"/>

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Empty space

@@ -1574,6 +1574,7 @@
"Graphics_EffectsQ": "Effects Quality",
"Graphics_ShadingQ": "Shading Quality",
"Graphics_Distortion": "Distortion",
"Graphics_AnisotropicSampling": "Anisotropic Sampling",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Technically this is already available under _GenshinGameSettingsPage, I think we can reuse it instead of telling localizer to resubmit exactly the same thing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants